This was easier than expected, MatrixOfConstraints and struct_of_constraints_by_set_types are great.

Weird fails are:

test_model_UpperBoundAlreadySet
test_model_LowerBoundAlreadySet

Probably due jump-dev/MathOptInterface.jl#1665

test_model_ScalarFunctionConstantNotZero

Possibly UniversalFallback stuff

cc @odow

Codecov Report

Merging #90 (73d5fea) into master (f0d2d91) will increase coverage by 3.10%.
The diff coverage is 89.94%.

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
+ Coverage   82.02%   85.12%   +3.10%     
==========================================
  Files          11       11              
  Lines        1391     1217     -174     
==========================================
- Hits         1141     1036     -105     
+ Misses        250      181      -69     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0d2d91...73d5fea. Read the comment docs.

test_model_UpperBoundAlreadySet
test_model_LowerBoundAlreadySet
test_model_ScalarFunctionConstantNotZero

Just exclude these ones for now. The AlreadySet ones will be fixed in an upcoming release of MOI. The ScalarFunctionConstantNotZero is optional.

Version >= 1.7 requires a fix in MOI: jump-dev/MathOptInterface.jl#1711